feat: add cursor overlay pipeline (depends on #207)#217
feat: add cursor overlay pipeline (depends on #207)#217EtienneLescot wants to merge 4 commits intosiddharthvaddem:mainfrom
Conversation
b94ffca to
a5d52d8
Compare
36081e4 to
e8e6a95
Compare
|
Just wanted to thank you for working on this and helping contribute to the project 🙏 I know this is a draft, but wanted to reiterate that I was never against Native Capture. We can have the cursor related settings hidden for Linux and Windows. If that helps. Let me know if there's any further clarification needed. Sorry if this feels like going back and forth Had a couple of folks on X reach out and say they really wanted this for macOS. No rush on this at all. Don't feel obliged. This is a pretty large change which would require a lot of testing to avoid bugs. This is what usually helps me (might have missed something): Capture + Launch
Audio
Editor Load + Playback
Timeline
Project Persistence
Other
|
|
@siddharthvaddem |
take your time 🙏 no rush. want to make sure it is implemented correctly even if it takes time. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
…and playback - Implement native bridge for Windows cursor capture via PowerShell/C# - Add cursor-free capture using getDisplayMedia with setDisplayMediaRequestHandler - Update video player and exporters to support native cursor telemetry - Enable system audio capture on Windows via WASAPI loopback - Add interpolation for smoother cursor movement in playback and export - Improve cursor scaling and visibility handling in editor and playback
ff8645e to
bc200cd
Compare
Description
This follow-up depends on #207.
GitHub cannot target
feat/recordly-cursor-pipelineas the PR base here because that branch exists only on my fork, so this PR still points atmain. In practice, the intended review order is:Included in this PR
Cursor overlay rendering
Cursor editor controls
Cursor project data
Main files
src/components/video-editor/SettingsPanel.tsxsrc/components/video-editor/VideoEditor.tsxsrc/components/video-editor/VideoPlayback.tsxsrc/components/video-editor/videoPlayback/cursorRenderer.tssrc/components/video-editor/videoPlayback/motionSmoothing.tssrc/components/video-editor/videoPlayback/uploadedCursorAssets.tsNotes
./node_modules/.bin/tsc --noEmitandnpm exec vite build